Skip to content

Fixed tree-sitter-norg parser builds on MacOS. - #1587

Merged
benlubas merged 1 commit into
nvim-neorg:mainfrom
pcekm:main
Mar 28, 2025
Merged

Fixed tree-sitter-norg parser builds on MacOS.#1587
benlubas merged 1 commit into
nvim-neorg:mainfrom
pcekm:main

Conversation

@pcekm

@pcekm pcekm commented Sep 29, 2024

Copy link
Copy Markdown
Contributor
  • Added use_makefile, which is how nvim-treesitter builds the norg parser by default and which correctly adds the required compiler flags
  • Removed a macOS build kludge that is no longer needed (and seems not to work anymore)
  • Removed branch specification which is ignored when setting revision.

Tested on an Apple silicon machine running macOS 14.7, and an Intel machine running macOS running macOS 13.7.

Fixes nvim-neorg/tree-sitter-norg#45

 - Added use_makefile, which is how nvim-treesitter builds the norg
   parser by default and correctly adds the required flags
 - Removed a MacOS build kludge that is no longer needed (and seems not
   to work anymore)
 - Removed branch specification which is ignored when setting revision.
@zerachiel016

Copy link
Copy Markdown

Hey I think you're changes are already covered in this PR

@benlubas

benlubas commented Oct 5, 2024

Copy link
Copy Markdown
Contributor

#1020 is stalled though, I could see merging this in the meantime.

@wjyoung65

Copy link
Copy Markdown

FWIW, I got a clean build with this PR on an M3 MacBook running MacOS 15, and also on an x86_64 laptop running Linux Mint.

@saxon1964

saxon1964 commented Mar 25, 2025

Copy link
Copy Markdown

This is a must have! Compiled without issues on Neorg 9.2.0 and Intel MacOS 14.7.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error during compilation

5 participants